Skip to content

Conversation

@SirPyTech
Copy link

Steps:

  1. Create a Manager of Operating Units
  2. Create some Operating Units
  3. Log in as the created Manager
  4. Create an Invoice
  5. Click on the Operating Unit field

Before this change:
Not all Operating Units can be selected.

After this change:
All Operating Units can be selected.

@SirPyTech SirPyTech force-pushed the 14.0-fix-operating_unit-assign_to_manager branch from 4e1698e to 55d63a1 Compare December 31, 2025 13:49
Steps:
1. Create a Manager of Operating Units
2. Create some Operating Units
3. Log in as the created Manager
4. Create an Invoice
5. Click on the Operating Unit field

Before this change:
Not all Operating Units can be selected.

After this change:
All Operating Units can be selected.
@SirPyTech SirPyTech force-pushed the 14.0-fix-operating_unit-assign_to_manager branch from 55d63a1 to 8b23d50 Compare December 31, 2025 14:00
@SirPyTech SirPyTech marked this pull request as ready for review December 31, 2025 14:05
@StefanRijnhart StefanRijnhart changed the title [FIX] account_operating_unit: Manager can select all OUs [14.0][FIX] account_operating_unit: Manager can select all OUs Jan 5, 2026
Copy link

@Marco-ooops Marco-ooops left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test funzionale ok

Copy link
Contributor

@HekkiMelody HekkiMelody left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code review: LGTM

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@francesco-ooops
Copy link
Contributor

@AaronHForgeFlow what do you think? Code and functional approved, we can fw-port

@francesco-ooops
Copy link
Contributor

@AaronHForgeFlow can we merge this? thanks!

Copy link
Contributor

@AaronHForgeFlow AaronHForgeFlow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok. Code review LGTM. I think the forward port is not needed, such domains do not exist in v15.

@AaronHForgeFlow
Copy link
Contributor

/ocabot merge minor

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 14.0-ocabot-merge-pr-824-by-AaronHForgeFlow-bump-minor, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 58a0b55 into OCA:14.0 Jan 13, 2026
7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 45f8e48. Thanks a lot for contributing to OCA. ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants